feat: add publish status to route in manager api #450
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
3 tasks
liuxiran
force-pushed
the
route_publish_api
branch
from
b747828 to
ae2a1b1
Compare
|
nice work @liuxiran 👍 |
5 tasks
juzhiyuan
reviewed
Sep 9, 2020
liuxiran
force-pushed
the
route_publish_api
branch
from
9352b53 to
5e2c54a
Compare
|
@liuxiran CI failed, please take a look |
fixing :) |
3 tasks
|
the reason for ci failed is the same with #452 (comment) , we updated the logic of routeGroup. this pr is depend on pr #460 @juzhiyuan so sorry for the interdependence between these prs. I'll pay attention next time~! |
|
@liuxiran ok, so just waiting for others' review :DD |
thanks for your time~ |
nic-chen
requested changes
Sep 10, 2020
|
api ci not pass |
@nic-chen maybe this reason? |
maybe.so we should handle that pr first. |
liuxiran
force-pushed
the
route_publish_api
branch
from
5d7e3a1 to
3a80623
Compare
liuxiran
force-pushed
the
route_publish_api
branch
from
3a80623 to
78324a1
Compare
nic-chen
reviewed
Sep 14, 2020
| "paths":["/*"], | ||
| "methods":["GET","HEAD","POST","PUT","DELETE","OPTIONS","PATCH"], | ||
| "status":false, | ||
| "status":true, |
There was a problem hiding this comment.
'offline' is a PUT type API, so create a route online first, then we could do offline :)
nic-chen
approved these changes
Sep 15, 2020
LiteSun
pushed a commit
to LiteSun/incubator-apisix-dashboard
that referenced
this pull request
Sep 21, 2020
* feat: add publish status to route in manager api * fix: update publish and offline route * feat: add route publish test module * fix: synchronize content for route publish/offline
juzhiyuan
added a commit
that referenced
this pull request
Sep 22, 2020
* feat: preview pluginchart (#456) * feat: added pluginchart preview in route step4 * feat: change PluginChart to PluginOrchestration * add tools to check ASF headers in code files. (#454) * chore: remove unnecessary license file (#462) * remove unnecessary license file * remove unnecessary license file for `api7 dashboard` * Increase the checking of .conf and .toml files. And delete the redundant configuration of the ASF-Release.cfg file. (#461) * Increase the checking of .conf and .toml files. * Skip the check of dag-to-lua-1.1/README.md file * Put the "run Makefile" check before "get lua lib". * Put the "run Makefile" check before "get lua lib". * Put the "run Makefile" check before "get lua lib". Co-authored-by: lin <lin@MBPro.local> * ci: Add lint (#455) Co-authored-by: Rapiz <code@rapiz.me> * Feat: dashboard support route group (#433) * feat: route group UI * feat(route-ui): add routegroup when create route * fix: add routegroup to route list * fix: support add routegroup together with route * fix: route path define error * feat: trigger redeploy * fix: update i18n key Co-authored-by: 琚致远 <juzhiyuan@apache.org> * feature:Independent license check operation file. (#465) * feat: support ungroup in manager api (#460) * feat: support ungroup in manager api * fix: synchronize route group to route * fix: update test case * feat: add publish status to route in manager api (#450) * feat: add publish status to route in manager api * fix: update publish and offline route * feat: add route publish test module * fix: synchronize content for route publish/offline * feat(route): add publish and offline to route (#451) * feat(route): add publish and offline to route * feat: trigger redeploy * feat: trigger redeploy * Update Create.tsx * fix: update refer to the code review * fix: init status default value * fix: add a help msg to status Form.Item Co-authored-by: 琚致远 <juzhiyuan@apache.org> * feat: remove netlify (#472) * fix: host should not be required (#477) * feat: added new plugin dependency (#475) * feat: added new plugin dependency * feat: omit shadow var * feat: omit shadow var * fix: host should not be required (#479) * fix: host should not be required * feat: remove required rule * feat: Route debug (#485) * feat: new api get route and apisix url * feat: online debug * fix: update refer to the review, fix logical error as well * fix: rename getRouteWithApisixUrl * feat: update plugin (#482) * feat: improve route (#483) * feat: improve step1 * feat: improve step2 * feat: improve createStep4 * feat: improve transform * fix: event loop * feat: clean code * fix: lost route_group info when enable redirect * feat: UI improve * fix: checkHostWithSSL with empty hosts * Update Create.tsx Co-authored-by: Firstsawyou <52862365+Firstsawyou@users.noreply.github.com> Co-authored-by: nic-chen <33000667+nic-chen@users.noreply.github.com> Co-authored-by: lin <lin@MBPro.local> Co-authored-by: Rapiz <contact@rapiz.me> Co-authored-by: Rapiz <code@rapiz.me> Co-authored-by: liuxiran <belovedxixi@126.com> Co-authored-by: 琚致远 <juzhiyuan@apache.org>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Please answer these questions before submitting a pull request
Why submit this pull request?
Bug fix
New feature provided
Improve performance
Related issues
issue Feature：route publish management #437 manage api
Bugfix
Description
How to fix?
New feature or improvement
add a Column to route, called
statusstatus = true, means create a new route in both mysql and apisix;status=false: means create a route only in mysqlmodified the logic of editing and deleting route at the same time,
add manager apis to support change route
status